Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport security workflow to 2.4 branch #639

Merged

Conversation

bowenlan-amzn
Copy link
Member

@bowenlan-amzn bowenlan-amzn commented Dec 28, 2022

Issue #, if available:

Description of changes:
Backport #611 to 2.4 branch

CheckList:

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

* Unify wipe indices logic after tests

Signed-off-by: bowenlan-amzn <[email protected]>

* Enhance wipeAllIndices function

Signed-off-by: bowenlan-amzn <[email protected]>

* security workflow setup

Signed-off-by: bowenlan-amzn <[email protected]>

* add more settings

Signed-off-by: bowenlan-amzn <[email protected]>

* Rewrite SecureRestClientBuilder

Signed-off-by: bowenlan-amzn <[email protected]>

* For now, use local cluster

Signed-off-by: bowenlan-amzn <[email protected]>

Signed-off-by: bowenlan-amzn <[email protected]>
Signed-off-by: bowenlan-amzn <[email protected]>
@bowenlan-amzn bowenlan-amzn changed the title Backportsecurityto2.4 Backport security workflow to 2.4 branch Dec 28, 2022
@codecov-commenter
Copy link

codecov-commenter commented Dec 28, 2022

Codecov Report

Merging #639 (ae9112f) into 2.4 (ccbde24) will increase coverage by 0.33%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                2.4     #639      +/-   ##
============================================
+ Coverage     75.38%   75.72%   +0.33%     
- Complexity     2602     2614      +12     
============================================
  Files           330      330              
  Lines         15115    15115              
  Branches       2338     2338              
============================================
+ Hits          11395    11446      +51     
+ Misses         2389     2338      -51     
  Partials       1331     1331              
Impacted Files Coverage Δ
.../rollup/action/index/TransportIndexRollupAction.kt 72.09% <0.00%> (ø)
...anagement/indexstatemanagement/model/Transition.kt 61.64% <0.00%> (-4.11%) ⬇️
...exmanagement/opensearchapi/OpenSearchExtensions.kt 75.65% <0.00%> (-0.87%) ⬇️
...arch/indexmanagement/rollup/RollupSearchService.kt 61.11% <0.00%> (ø)
...agement/indexstatemanagement/ManagedIndexRunner.kt 47.13% <0.00%> (ø)
...nt/indexstatemanagement/ManagedIndexCoordinator.kt 70.84% <0.00%> (+1.16%) ⬆️
...rch/indexmanagement/rollup/model/RollupMetadata.kt 86.12% <0.00%> (+1.73%) ⬆️
...ent/indexstatemanagement/util/ManagedIndexUtils.kt 77.97% <0.00%> (+2.20%) ⬆️
...management/indexstatemanagement/MetadataService.kt 64.92% <0.00%> (+2.23%) ⬆️
.../action/explain/TransportExplainTransformAction.kt 73.03% <0.00%> (+2.24%) ⬆️
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bowenlan-amzn bowenlan-amzn marked this pull request as ready for review December 29, 2022 01:14
@bowenlan-amzn bowenlan-amzn requested a review from a team December 29, 2022 01:14
@bowenlan-amzn bowenlan-amzn merged commit 2e0b8b6 into opensearch-project:2.4 Jan 3, 2023
@bowenlan-amzn bowenlan-amzn deleted the backportsecurityto2.4 branch January 3, 2023 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants